Skip to content
This repository has been archived by the owner on Oct 18, 2024. It is now read-only.

feat: ✨ changed error response to success flag #134

Merged
merged 3 commits into from
Apr 24, 2024

Conversation

jordany33
Copy link
Contributor

re #115

@jordany33 jordany33 temporarily deployed to staging-134 February 29, 2024 01:43 — with GitHub Actions Inactive
@jordany33 jordany33 temporarily deployed to staging-134-docs February 29, 2024 01:43 — with GitHub Actions Inactive
Copy link
Member

@ecxyzzy ecxyzzy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for my late response.

Just adding success: boolean to BaseResponse doesn't actually work, as the checks show, because RawResponse<T> needs to be a tagged union. Should be a relatively straightforward fix, but let me know if you have any issues.

@ecxyzzy ecxyzzy linked an issue Mar 9, 2024 that may be closed by this pull request
@jordany33 jordany33 temporarily deployed to staging-134-docs April 18, 2024 21:42 — with GitHub Actions Inactive
@jordany33 jordany33 temporarily deployed to staging-134 April 18, 2024 21:42 — with GitHub Actions Inactive
@jordany33 jordany33 temporarily deployed to staging-134 April 18, 2024 21:50 — with GitHub Actions Inactive
@jordany33 jordany33 temporarily deployed to staging-134-docs April 18, 2024 21:50 — with GitHub Actions Inactive
Copy link
Member

@ecxyzzy ecxyzzy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for working on this!

@ecxyzzy ecxyzzy merged commit 4bbd53d into main Apr 24, 2024
5 checks passed
@ecxyzzy ecxyzzy deleted the 115-rest-response-type-to-tagged-union branch April 24, 2024 17:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Convert REST response type to tagged union
2 participants